Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update IssueId type to long #3

Closed
wants to merge 231 commits into from

Conversation

snirburkush
Copy link

breaking change from 3 days ago:
octokit#2890

closes: https://apiiro.atlassian.net/browse/LIM-20707

dependabot bot and others added 30 commits November 9, 2020 08:55
Bumps [Microsoft.NET.Test.Sdk](https://github.com/microsoft/vstest) from 16.7.1 to 16.8.0.
- [Release notes](https://github.com/microsoft/vstest/releases)
- [Commits](microsoft/vstest@v16.7.1...v16.8.0)

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Used ConcurrentDictionary in the not ThreadSafeDictionary
    
I'm taking a break. I'll enable automated dependency updates when I'm back in 2021. But probably not with Dependabot, but Renovate.
This reverts commit f504167.

Sorry I missed to filter out this repository when I run my script :(
The previous link redirected to https://github.com/octokit/octokit.net/issues?q=label%3Aup-for-grabs, which would also show closed issues; the new link only shows open issues.
Update supported .NET Framework and .NET Standard  text
PullRequestCommit Author and Committer fields were mistakenly typed with
Committer when they should be of type User.

Previously the types of properties were:

* `PullRequestCommit.Author/Committer`: `Committer`
* `Commit.Author/Committer`: `Committer`

Correct types should be:

* `PullRequestCommit.Author/Committer`: `User`
* `Commit.Author/Committer`: `Committer`

These fields always fail to deserialize in the API calls but produce no
errors, only objects with default values.
Allows you to search for issues with no project.
* enable windows target for building
* workaround IGNORE_NORMALISATION_GIT_HEAD_MOVE reported by GitVersion
Bumps [Microsoft.NET.Test.Sdk](https://github.com/microsoft/vstest) from 16.8.0 to 16.8.3.
- [Release notes](https://github.com/microsoft/vstest/releases)
- [Commits](microsoft/vstest@v16.8.0...v16.8.3)

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Bump Codecov from 1.12.3 to 1.12.4 in /build

Bumps [Codecov](https://github.com/codecov/codecov-exe) from 1.12.3 to 1.12.4.
- [Release notes](https://github.com/codecov/codecov-exe/releases)
- [Changelog](https://github.com/codecov/codecov-exe/blob/master/Changelog.md)
- [Commits](codecov/codecov-exe@1.12.3...1.12.4)

Signed-off-by: dependabot[bot] <[email protected]>

* fix path to codecov executable

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Brendan Forster <[email protected]>
When serializing the `NewDeployment` type, the `Payload` is serialized as an escaped string because JSON.NET doesn't know it's meant to be JSON.

This causes a problem when you call the API because the Payload is supposed to be a JSON dictionary that's just part of the overall payload. It's not supposed to be an escaped string.

That's why the JSON deserializer fails on it. Not only that, any deployments created using the current Octokit.net will create an invalid payload.

This PR fixes it by changing the type of `Payload` to a dictionary. THIS IS A BREAKING CHANGE, but the old behavior was broken so it forces a new correct behavior.

Fixes octokit#2250
* Move the build to .net 5

.NET 5 is fine for building projects that target older versions of .NET. This also should fix the AppVeyor build which fails because the latest `coverlet.console` only supports .NET 5.

* Some things need .NET core 3.1 available

Co-authored-by: Brendan Forster <[email protected]>
Bumps [Cake.Coverlet](https://github.com/romanx/Cake.Coverlet) from 2.5.1 to 2.5.4.
- [Release notes](https://github.com/romanx/Cake.Coverlet/releases)
- [Commits](Romanx/Cake.Coverlet@v2.5.1...v2.5.4)

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
TalfinApiiro and others added 29 commits February 27, 2023 19:14
This reverts commit 49e1637.
This reverts commit c74a17c.
This reverts commit 727d9e0.
Necessary changes to fully support response caching
…nt-support-audit-log-calls

Extend GitHub client support Organization audit log call
…ate-method

Fix get user last activity date method
…hub-branch-ruleset-rules

Add support for fetching GitHub repository branch ruleset rules
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.